This looks great. Could you add a test for this, so it doesn't get
regressed?

On Thu, Sep 5, 2013 at 6:05 AM, tan4eto86 notifications@github.com wrote:

Update hasmany association with through option

---

You can merge this Pull Request by running

git pull https://github.com/tan4eto86/model master

Or view, comment on, or merge it at:

#99
Commit Summary

• Update hasmany association with through option to work when one of
  instances has more than one mapped instanses of the second model.

File Changes

• M lib/index.js https://github.com/mde/model/pull/99/files#diff-0(5)

Patch Links:

• https://github.com/mde/model/pull/99.patch
• https://github.com/mde/model/pull/99.diff